Experiment - #127
Open
DmitrievDmitriyA wants to merge 16 commits into
Open
Conversation
…nts; add map extractor
…-id DSL + per-language tsv + chrome/stages); regenerate EN/RU/ZH, all pass mapcheck
…are no longer auto-laid-out. Each hint carries a polar offset (angle deg, dist px) from its target in structure.dsl; extract.py fills these from the hand map so notes replay verbatim, then they're tuned until mapcheck is clean. Drops the iterative margin-push / collision engine. Restructure: the canonical source moves examples/fullmap -> roadmap (it is the real map, not a demo), and the feature-demo examples/ are removed. Regenerates all three live maps from the source; mapcheck reports 0 hard errors and 0 cross-language drift.
…xtract.py and remap.py are one-off helpers (derive roadmap/ from the hand-drawn maps, onboard a new language) — not part of the build loop. Move them into tools/mapgen/bootstrap/ with their own README, and slim the main README to a pointer: the DSL is the source of truth going forward. Document the hint polar convention in structure.dsl (angle: 0=right, 90=up, 180=left, 270=down; dist in pixels) and have extract.py emit that comment so a re-extract keeps it.
…ide Port build.ps1 to build.py (Pillow text metrics, same lib as mapcheck, so generator and checker agree on widths). Node widths land within ~1.5px of the PowerShell build; re-tuned 5 RU hints to clear residual sub-12px overlaps. Remove build.ps1. Add `arrow=left|right|top|bottom` to hints (which box edge the arrow starts from) and default it to the edge facing the target on the dominant axis, so notes above/below a target arrow from top/bottom instead of the side. Regenerate all three maps; mapcheck clean. Update README/.gitignore/bootstrap docs to build.py; drop TODO -1 / -1.1.
DmitrievDmitriyA
force-pushed
the
experiment
branch
from
July 31, 2026 07:43
bd03158 to
8e7dae9
Compare
salmer
requested changes
Jul 31, 2026
| @@ -0,0 +1,41 @@ | |||
| name: Check maps | |||
Owner
There was a problem hiding this comment.
Could you "Description" with a short description what the workflow does
| - wording (a label, a hint, the date) → the matching row in `en.tsv` / `ru.tsv` / `zh.tsv` | ||
| (a new node needs a row in **all three**). | ||
| 2. Rebuild and copy over the maps (needs Python + Pillow + the draw.io desktop CLI): | ||
| ```bash |
Owner
There was a problem hiding this comment.
What about to write setup script which can be used either by us or by contrubutors? Then they/we just run it from the repo, and the magic happens
| @@ -0,0 +1,205 @@ | |||
| # mapgen — generate draw.io map fragments from a text DSL + translations | |||
|
|
|||
Owner
There was a problem hiding this comment.
Can we add some screenshots maybe of somebody wants to add new leafs/etc?
Usually visual content works better in addition of the text
| @@ -0,0 +1,72 @@ | |||
| # mapgen/bootstrap — reverse-engineer a hand-drawn map into the DSL | |||
|
|
|||
Owner
There was a problem hiding this comment.
Extra pictures of the map DSL design etc? To improve readability and understanding of it's design.
Maybe extra diagrams to show high-level design of DSL/generator?
| @@ -0,0 +1,358 @@ | |||
| #!/usr/bin/env python3 | |||
| """ | |||
| mapcheck — validate the roadmap maps (<Lang>/Graph/roadmap.drawio.svg). | |||
Owner
There was a problem hiding this comment.
It is completely Ai-generated, right?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Please include a summary of the change and which issue is fixed.
Checklist
.lycheeignorewith a comment).lycheelocally to verify links according toCONTRIBUTING.md.